feat(Upgrade): mandatory cross-recommendation interference analysis (Step 6b)#1442
Closed
82ATW wants to merge 1 commit into
Closed
feat(Upgrade): mandatory cross-recommendation interference analysis (Step 6b)#144282ATW wants to merge 1 commit into
82ATW wants to merge 1 commit into
Conversation
…Step 6b) Prior-Status gating checks each recommendation against the PAST; Step 6b checks them against EACH OTHER. When a report carries more than 10 recommendations, a pairwise interference sweep (measurement confounds, same-file collisions, input dependencies, doctrine bundling) becomes mandatory, adversarially verified by a second agent, and emitted as a new 'Interactions & Sequencing' report section with a conflict table, wave-ordered execution sequence, and synergies. Retargets the orphaned PR danielmiessler#1340 (which patched the frozen Releases/v5.0.0 snapshot) onto the live skill at LifeOS/install/skills/Upgrade/.
Owner
|
Thanks @82ATW — genuinely sharp observation that Prior-Status gates each recommendation against the past but nothing gated them against each other. We went a different way and are not adopting it as a mandatory step, but it was a good idea, well designed (the interference taxonomy and the >10-rec gate especially). This repo is generated from our source tree each release, so we close rather than merge here. Thanks for engaging with the Upgrade skill this concretely. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds a mandatory cross-recommendation interference analysis (Step 6b) to the Upgrade skill. Prior-Status gating checks each recommendation against the PAST; Step 6b checks them against EACH OTHER.
When a report carries >10 recommendations, the skill must:
⚠️ Interactions & Sequencingsection — conflict table + wave-ordered execution sequence + synergies.At ≤10 recommendations the section reduces to a single line, so quick modes stay fast.
Why
A 44-recommendation report shipped with every row Prior-Status-gated but zero pairwise analysis; a retrofit found two high-severity measurement confounds (a fallback setting and a safety-routing behavior each silently swapping the model under an A/B eval) plus six same-file collisions. A large recommendation set is itself a system — this gates the set against itself.
Files
LifeOS/install/skills/Upgrade/Workflows/Upgrade.md— new Step 6b (gate, matrix classes, adversarial pass, guards) + Step 7 section orderLifeOS/install/skills/Upgrade/References/OutputFormat.md— new Interactions & Sequencing section spec + hard rule 11LifeOS/install/skills/Upgrade/SKILL.md— Step 6b gate in extraction rules, section order, GotchaLifeOS/install/skills/Upgrade/References/ExampleReport.md— example of the ≤10 single-line formNotes
Supersedes #1340, which targeted the frozen
Releases/v5.0.0snapshot before the v6.0.0 restructure moved the live skill toLifeOS/install/skills/Upgrade/. Same feature, retargeted to the live files with current naming.